Skip to content

docs: tighten gopher padding and link upstream in README subtitle#32

Merged
millerjp merged 2 commits intomainfrom
docs/gopher-padding-and-link
Apr 21, 2026
Merged

docs: tighten gopher padding and link upstream in README subtitle#32
millerjp merged 2 commits intomainfrom
docs/gopher-padding-and-link

Conversation

@millerjp
Copy link
Copy Markdown

Summary

Two small polish items:

  1. Tighter gopher padding. The previous README logo had ~24% padding per side — considerably looser than mask's ~10% convention, which made the gopher look smaller than it should. Re-cropped the source gopher to its bounding box and re-padded to match mask's content-height ratio (93% of frame). Regenerated `social-preview-1280x640.png` from the tightened logo so the card inside the preview has the same visual weight as the README.

  2. Clickable upstream link. The README subtitle now links `rgooding/go-syncmap` directly to https://github.com/rgooding/go-syncmap so readers can jump to the upstream with a single click.

Validation

  • `make check` — green, coverage 100%.
  • `TestLLMs_FullTxtIsUpToDate` — passes after regenerating `llms-full.txt`.
  • Visual comparison: syncmap logo now has the same tight crop as mask's at the same 128px display width.

Post-merge

Desktop copy of the updated `social-preview-1280x640.png` is at `~/Desktop/syncmap-social-preview-1280x640.png` — re-upload via Settings → Social preview if you've already uploaded the earlier version.

Two small README polish items following the fork-framing PR:

  * Logo padding was 24% per side — too loose against mask's
    ~10% convention. Re-cropped the source gopher to its
    bounding box and padded to match mask's content-height
    ratio (93% of frame), so the gopher fills the frame with
    similar visual weight to axonops/mask/.github/images/logo-readme.png.
    Re-rendered social-preview-1280x640.png from the tightened
    logo so the card inside the preview matches too.

  * README subtitle now hyperlinks `rgooding/go-syncmap`
    directly to https://github.com/rgooding/go-syncmap so
    readers can one-click through to the upstream.

No code change. Coverage remains 100%; llms-full.txt not affected
(README subtitle reflows inside the bundle but go doc -all is the
canonical source; re-running the drift guard will confirm after
the merge).
  * Logo: re-cropped the source gopher to its bounding box and
    repadded to match axonops/mask's ~10% convention (the previous
    layout had ~24% padding per side and the gopher looked small).
    Regenerated social-preview-1280x640.png from the tightened
    logo so the card inside the preview has the same visual weight.
  * README subtitle now links `rgooding/go-syncmap` directly to
    https://github.com/rgooding/go-syncmap.
  * llms-full.txt regenerated to pick up the README change.

No code change. Coverage remains 100%.
@millerjp millerjp merged commit 7476de1 into main Apr 21, 2026
@millerjp millerjp deleted the docs/gopher-padding-and-link branch April 21, 2026 05:49
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant